Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

test(mirage): correct server authority string in preview builds (backport #1283) #1285

Merged
merged 1 commit into from
Jun 12, 2024

Conversation

mergify[bot]
Copy link

@mergify mergify bot commented Jun 12, 2024

Welcome to Cryostat! 👋

Before contributing, make sure you have:

  • Read the contributing guidelines
  • Linked a relevant issue which this PR resolves
  • Linked any other relevant issues, PR's, or documentation, if any
  • Resolved all conflicts, if any
  • Rebased your branch PR on top of the latest upstream main branch
  • Attached at least one of the following labels to the PR: [chore, ci, docs, feat, fix, test]
  • Signed all commits using a GPG signature

To recreate commits with GPG signature git fetch upstream && git rebase --force --gpg-sign upstream/main


Fixes: #1271

How to manually test:

  1. Check out PR
  2. yarn start:dev:preview should bring up the preview/demo UI. There should be a Fake Target, with a Demo template. Topology view should work, dashboard should work, recording start/stop/archive/delete should work, expanding recording for automated analysis should work. GraphQL-powered things will not work, snapshot recording creation does not work.
  3. yarn run build:preview:notests && cd dist && python3 -m http.server, then open http://localhost:8000 in a browser. The preview should work just the same as in the step above.

This is an automatic backport of pull request #1283 done by [Mergify](https://mergify.com).

* test(mirage): set server authority string in preview builds

* fix(mirage): ensure wsUrl matches that in application code

---------

Co-authored-by: Thuan Vo <thuan.votann@gmail.com>
(cherry picked from commit 3aaeead)
@github-actions github-actions bot added the needs-triage Needs thorough attention from code reviewers label Jun 12, 2024
Copy link

Hi @mergify[bot]! Add at least one of the required labels to this PR

Required labels are : chore,ci,cleanup,docs,feat,fix,perf,refactor,style,test

@andrewazores andrewazores added test safe-to-test and removed needs-triage Needs thorough attention from code reviewers labels Jun 12, 2024
@andrewazores andrewazores merged commit 8265836 into cryostat-v3.0 Jun 12, 2024
24 of 25 checks passed
@mergify mergify bot deleted the mergify/bp/cryostat-v3.0/pr-1283 branch June 12, 2024 18:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant